Ml Autoscaling Named Writables fix#65392
Conversation
Named writeables were not registered.
|
Pinging @elastic/ml-core (:ml) |
|
This was dropped during the backport indeed. But the reason is different. In 7.x, we need to register named writeables via |
|
Given that autoscaling is still experimental, I think we should not move things to xpack-core ahead of time. I have put it on our autoscaling sync for next week to discuss this. I would be inclined to just document that autoscaling does not work with the transport client. Moving the specific ML class would require moving other parts of autoscaling to xpack-core too. And for it to be functional, actions, requests and responses have to move too. So for now, I would prefer to get this PR in as is to enable consumers of the API to start testing ML autoscaling and then we can tackle the move to xpack core (if that is the decision taken) in a follow-up. Would that be ok? |
dimitris-athanasiou
left a comment
There was a problem hiding this comment.
LGTM Your point totally makes sense.
|
Thanks Dimitris! |
Named writeables were not registered.
This looks like it was a backport failure when backporting #59309 in #65151, since this is the first named writable registered by the ml module in 7.x.